Skip to content
This repository has been archived by the owner on Dec 15, 2021. It is now read-only.

IS-1120: prevalidate a tx more strictly 1 #524

Closed
wants to merge 4 commits into from
Closed

IS-1120: prevalidate a tx more strictly 1 #524

wants to merge 4 commits into from

Conversation

endlessgate
Copy link
Contributor

@endlessgate endlessgate commented Oct 12, 2020 •
edited
Loading

Pre-validation with strict for entered transaction request

Update

  • Version check
    • Support v3 protocol only
    • Field check
  • Int-type check
    • Not allow uppercase character in hex string
    • Malformed number string: "0x03", "0x0003", "3"
  • Address-type check
    • Malformed address format: "hx1234", "0xqw1234", "cxFEAC"
    • Invalid format check
  • Required parameters check

@endlessgate endlessgate requested a review from a team October 12, 2020 06:50
@endlessgate endlessgate self-assigned this Oct 12, 2020
@cow-hs cow-hs changed the base branch from master to develop October 13, 2020 00:11
@cow-hs cow-hs closed this Oct 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees

@endlessgate endlessgate

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@endlessgate @cow-hs @goldworm-icon